Skip to content

Edits suggested by Rutuja #16

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Sep 27, 2024
Merged

Edits suggested by Rutuja #16

merged 2 commits into from
Sep 27, 2024

Conversation

Rutuja-desh
Copy link
Collaborator

Hi, suggested some grammatical errors. Please go through them and let me know if you need anything else.
Thanks!

Copy link

github-actions bot commented Sep 27, 2024

PR Preview Action v1.4.8
Preview removed because the pull request was closed.
2024-09-27 17:15 UTC


=== Red Hat Partners

Partners should be able to access the new https://partner.demo.redhat.com[Red Hat Demo Platform for Partners,window=read-later] by logging in with their RHN account credentials. For more information on this you can refer https://content.redhat.com/us/en/product/cross-portfolio-initiatives/rhdp.html#tabs-333fa7ebb9-item-b6fc845e73-tab[about Red Hat Demo Platform (RHDP),window=read-later]

Catalog link - ##FIX THIS##
// Can you fix the above link?
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not yet available. Waiting on Ashish

@@ -22,18 +22,20 @@ The PTL team acknowledges the valuable contributions of the following Red Hat as
This introductory course has a few, simple hands-on labs. You will use the https://demo.redhat.com/catalog?item=babylon-catalog-prod/equinix-metal.eqx-blank.prod&utm_source=webapp&utm_medium=share-link.ocp4-workshop-rhods-base-aws.prod[Equinix Metal baremetal blank server,window=read-later] catalog item in the Red Hat Demo Platform (RHDP) to run the hands-on exercises in this course.

Update the Catalog link - ##FIX THIS##
// Can you fix the above link?
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not yet available. Waiting on Ashish


When ordering this catalog item in RHDP:

. Select *Practice/Enablement* for the *Activity* field.
. Select the nearest or any region in the *Region* field.
. Select *n3.xlarge.x86* as the flavor in *Type* field.
. Select *n3.xlarge.x86* as the flavor in the *Type* field.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay

@@ -44,8 +46,10 @@ For this course, you should have:
* Red Hat Certified Systems Administrator certification, or equivalent knowledge of Linux system administration is recommended for all roles.
* https://rol.redhat.com/rol/app/courses/do280-4.14[Red Hat OpenShift Administration II: Configuring a Production Cluster (DO280),window=read-later], or equivalent knowledge on configuring a production Red Hat OpenShift cluster.
* https://rol.redhat.com/rol/app/technical-overview/do016-4.14[Red Hat OpenShift Virtualization (OCP Virt) Technical Overview (DO016),window=read-later]
// Above course showed - temporarily unavailable for me. Can you please check?
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I get that message but within few seconds, I can see it.

* https://rol.redhat.com/rol/app/courses/do316-4.14[Managing Virtual Machines with Red Hat OpenShift Virtualization (DO316),window=read-later], or equivalent knowledge on how to create virtual machines on Red Hat OpenShift cluster using Red Hat OpenShift Virtualization.
* Knowledge on installing Red Hat Advanced Cluster Management (RHACM) operator
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no link for this.

@@ -95,7 +95,7 @@ image::tenant_console_nodejs_app_page_success.png[]

== Test the High Availability and Resilience

. Find out the where the application pod is running.
. Find out where the application pod is running.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay.

@@ -107,21 +107,21 @@ nodejs-basic-6d55569c9c-gps2d 1/1 Running 0 2m51s 10.130.0.

. In this case, the application pod is running on the `tcn2.lab.example.com` node.
+
This means that the `tcn2.lab.example.com` node is running on `sno2` _Infrastructure_ cluster.
This means that the `tcn2.lab.example.com` node is running on the `sno2` _Infrastructure_ cluster.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay.


. To test high availability and resilience, first shut down the `sno2` VM, which will shutdown the _Infrastructure_ cluster.
+
image::tenant_console_nodejs_app_shutdwon_vm.png[]
+
Ensure that `sno2` VM is in shutoff state.
Ensure that the `sno2` VM is in a shutoff state.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay.

+
image::tenant_console_nodejs_app_shutdwon_vm_1.png[]

. If you refresh the application page, you will note the failure message again.
+
image::tenant_console_nodejs_app_page_failure.png[]

. The `pod-eviction-timeout` and `node-monitor-grace-period` parameters have the default value of `5m` and `40s` respectively.
. The `pod-eviction-timeout` and `node-monitor-grace-period` parameters have the default values of `5m` and `40s` respectively.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay.

. This test shows that even if one infrastructure node is down; the application automatically migrates to other infrastructure node.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay.

@sarveshpandit sarveshpandit merged commit c85fb8b into main Sep 27, 2024
1 check passed
@sarveshpandit sarveshpandit deleted the rutuja-edits branch September 27, 2024 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants